-
Notifications
You must be signed in to change notification settings - Fork 2
View proposal with tiptap docs #484
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
Conversation
7ab53fe to
98f1a19
Compare
98f1a19 to
815c7ea
Compare
bf35db9 to
da6af23
Compare
da6af23 to
b799719
Compare
b799719 to
0572e9f
Compare
| existingProposal?.id, | ||
| existingProposal?.proposalData, | ||
| ]); | ||
| return `proposal-${instance.id}-${existingProposal?.id ?? crypto.randomUUID()}`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We shouldn't end up in this state, but if we handle it by creating a new doc.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll use #492 when merged
scazan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good
Summary
Follow-up to #482. Renders proposal content from TipTap document server instead of legacy
descriptionfield.DocumentNotAvailableempty state when content fails to loadDemo
Success vs error state of document loading